Skip to content

Fix rust build cache github actions#213

Open
ChristianPavilonis wants to merge 2 commits intomainfrom
fix/github-actions-build-cache
Open

Fix rust build cache github actions#213
ChristianPavilonis wants to merge 2 commits intomainfrom
fix/github-actions-build-cache

Conversation

@ChristianPavilonis
Copy link
Collaborator

@ChristianPavilonis ChristianPavilonis commented Jan 29, 2026

First run: Rust tests ~5min run
Second run: Rust tests ~1min run

@ChristianPavilonis ChristianPavilonis force-pushed the fix/github-actions-build-cache branch from e4c644a to a7f7d4a Compare January 29, 2026 14:48
@ChristianPavilonis ChristianPavilonis changed the title DRAFT: fix rust build cache github actions Fix rust build cache github actions Jan 29, 2026
@ChristianPavilonis ChristianPavilonis marked this pull request as ready for review January 29, 2026 14:49
@ChristianPavilonis ChristianPavilonis force-pushed the fix/github-actions-build-cache branch from a7f7d4a to f244195 Compare January 29, 2026 14:50
Copy link
Collaborator

@aram356 aram356 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good overall but need to retain pinned Rust version

@ChristianPavilonis ChristianPavilonis force-pushed the fix/github-actions-build-cache branch from 5075497 to 12380f6 Compare February 4, 2026 21:52
Copy link
Collaborator

@aram356 aram356 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems we need rustup for wasm

components: "clippy, rustfmt"
cache-shared-key: cargo-${{ runner.os }}

- name: Add wasm32-wasi target
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔧 We need this because now the tests fail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants